snapcast: define ALSA support explicitly

As Hannu Nyman pointed out, in some cases the build of the package is
able to fail:

Package snapserver is missing dependencies for the following libraries:
libasound.so.2

He said: on some build runs, alsa is built (as required by other
packages), and if it is built before snapcast, it may be detected,
if the feature is not explicitly disabled in CMakeLists.txt.

Also: "I think that you could drop that AUDIO_SUPPORT conditional.
AUDIO_SUPPORT is selected e.g. if USB_SUPPORT is selected,
so it is not that big restriction."

Suggested-by: Hannu Nyman <[email protected]>
Signed-off-by: Szabolcs Hubai <[email protected]>

diff --git a/sound/snapcast/Makefile b/sound/snapcast/Makefile
index b097bad2f50ce747fca4d3b2e6954d52a6ee3eff..20b57b7f6109f2955dc8ea4cedf3e6959bacd6d6 100644 (file)
--- a/sound/snapcast/Makefile
+++ b/sound/snapcast/Makefile
@@ -21,6 +21,7 @@ include $(INCLUDE_DIR)/cmake.mk
 
 CMAKE_OPTIONS += \
        -DBUILD_TESTS=OFF \
+       -DBUILD_WITH_ALSA=ON \
        -DBUILD_WITH_AVAHI=OFF \
        -DBUILD_WITH_PULSE=OFF
 
@@ -28,7 +29,7 @@ define Package/snapcast/Default
   SECTION:=sound
   CATEGORY:=Sound
   TITLE:=Synchronous multiroom audio player
-  DEPENDS:=+AUDIO_SUPPORT:alsa-lib +libexpat +libatomic +libogg +libflac +libopus +boost +libsoxr
+  DEPENDS:=+alsa-lib +libexpat +libatomic +libogg +libflac +libopus +boost +libsoxr
   URL:=https://github.com/badaix/snapcast
 endef